-
Notifications
You must be signed in to change notification settings - Fork 954
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make CI green by skipping failing ruby-head and jruby-head tests #1162
base: master
Are you sure you want to change the base?
Conversation
What are you fixing? |
@albus522 the CI on master is not currently green. This PR aims to fix that. It seems that JRuby 9.3 is the source of failure... |
It will green up when jruby supports Rails 7. It is acceptably red for now. |
@albus522 the issues aren't related to Rails 7.0.0, as you can see it's failing on Rails 6.1.0 too. It's this bug: jruby/jruby#6984 |
I would suggest to merge this PR as it makes failures much more clear than what's on currently on DJ master. We should ignore failures on head etc. which master is not currently doing properly. |
This isn't fixing anything. It is just removing a bunch of stuff that is temporarily failing. Moving the the You can see the It is not green right now because jruby does not register as |
@albus522 in the link you just sent, I've updated the title of the PR to more accurately reflect the change here. |
I do not agree, failures should be red. The build finish step knows if the suite is considered passing. I agree that Github could make it more obvious and better distinguish allowed failures vs hard failures, but I will not be artificially marking everything green to get around a Github UI deficiency. |
Firstly, 100% agreed that Github should do a better job of marking skipped vs. failed (like Travis CI used to.) Unfortunately today Github does not. IMHO CI should indicate whether the code is stable for all currently supported versions -- |
Master isn't green right now. And I see I botched the release commit message this round. |
No description provided.